Skip to content

Implement TryFrom<SmallVec<T, N>> for [T; M] - #482

Merged
alejandro-vaz merged 2 commits into
servo:v2from
bolshoytoster:v2
Aug 29, 2026
Merged

Implement TryFrom<SmallVec<T, N>> for [T; M]#482
alejandro-vaz merged 2 commits into
servo:v2from
bolshoytoster:v2

Conversation

@bolshoytoster

Copy link
Copy Markdown
Contributor

See #357. Also deprecates into_inner, since this is a more general version of that function.

@alejandro-vaz

Copy link
Copy Markdown
Collaborator

oh yeah that is new, you have to run cargo fmt --all

it's just to standardize style

@bolshoytoster

bolshoytoster commented Aug 29, 2026

Copy link
Copy Markdown
Contributor Author

My IDE normally runs rustfmt on save, but I disable it when contributing to things, since usually it makes loads of unrelated changes.

@alejandro-vaz
alejandro-vaz added this pull request to the merge queue Aug 29, 2026
@alejandro-vaz

Copy link
Copy Markdown
Collaborator

looks good

Merged via the queue into servo:v2 with commit 89a0392 Aug 29, 2026
6 checks passed
@bolshoytoster

Copy link
Copy Markdown
Contributor Author

I meant to change into_inner to just be self.try_into(), but I was fumbling with git and forgot to re-add it. It shouldn't matter too much since it's deprecated anyway.

@alejandro-vaz

Copy link
Copy Markdown
Collaborator

no issue, we'll delete it anyway in 2.0.0-rc.1

alejandro-vaz pushed a commit to alejandro-vaz/rust-smallvec that referenced this pull request Aug 29, 2026
* Implement `TryFrom<SmallVec<T, N>> for [T; M]` and deprecate into_inner

* Run rustfmt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants